Skip to content

docs: add Aliyun Prometheus monitoring integration#38

Open
ysyneu wants to merge 1 commit intomainfrom
doc-review/2026-04-01-120000
Open

docs: add Aliyun Prometheus monitoring integration#38
ysyneu wants to merge 1 commit intomainfrom
doc-review/2026-04-01-120000

Conversation

@ysyneu
Copy link
Copy Markdown
Contributor

@ysyneu ysyneu commented Apr 1, 2026

Doc Review — 2026-04-01

Mode: diff (since 1 day ago)
Findings: 1 total — 1 high, 0 medium, 0 low

Changes

Finding Type Module Doc Action
f001 missing_coverage on-call/integration aliyun-prometheus.mdx Created new integration doc page (zh + en)

Details

New Aliyun Prometheus (阿里云 Prometheus 监控) alert integration was added to the platform:

  • Backend handler in fc-event (cmd/engine/controller/alert/aliyun_prometheus.go)
  • API route registered in fc-pgy
  • Frontend integration registered in fc-foundation-app (IntegrationCenter config, alert detail field mappings)

Documentation follows existing Aliyun integration patterns with:

  • Flashduty setup steps (private + shared integration)
  • Aliyun Prometheus webhook configuration steps
  • Severity mapping table (CRITICAL/ERROR→Critical, WARNING→Warning, INFO→Info)

Skipped findings

  • fc-event groupindex: Internal Redis-based grouping index optimization behind AB flag — not user-facing, no doc impact

Review checklist

  • Verify Aliyun Prometheus console navigation paths match actual UI
  • Check severity mapping matches current handler code
  • Ensure integration name matches what appears in Flashduty console

Mapping warnings

None

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant